New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
web-api: basic module system integration tests, removing unused bits, dependency updates #1724
Conversation
…or them, updated Busboy usage in tests to latest documented API.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM all around! Great to see the checks for both CJS and ESM 🎉
process.exit(1); | ||
} catch (e) { | ||
assert(e.message.includes('invalid_auth'), '❌ Did not receive expected "invalid_auth" response from `auth.test` API, ESM project integration test failed.'); | ||
console.log('✅ ESM project integration test succeeded!'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
}, | ||
"extends": "@tsconfig/recommended/tsconfig.json", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🙌 🙌
"typescript": "^4.1" | ||
"ts-node": "^10", | ||
"tsd": "^0.30.0", | ||
"typescript": "5.3.3" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🔒🔒🔒🔒🔒.3.3 👍 Happy to remove breaking builds from this!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Farewell, deno-shims/buffer-shim.js 👋 😔
Summary
Fixes #1679 .
In prep for web-api v7 release, doing some cleanup and dependency updates, specifically:
test/integration
: one for a CommonJS-based project and one for an ESM-based project, ensuring that no matter which module system is used, web-api is compatible with it. By changing thetype
field withinpackage.json
and/or modifying thetsconfig.json
compilerOptions
, the code emitted for distribution may or may not be compatible with each style of project. These integration tests ensure compatibility with both.typescript
dependency to the latest version. Locking to a specific version as typescript notoriously introduces breaking changes in minor and patch releases.tsconfig.json
and re-using the 'recommended' tsconfig base as much as possible.capture-console
dev dependency, can use sinon for the same functionality.